Skip to content

fix: correct spelling of "Rootstack" to "Rootstock" in README and his…#179

Merged
ivegabr merged 1 commit intorsksmart:mainfrom
aapsi:patch-1
Jun 16, 2025
Merged

fix: correct spelling of "Rootstack" to "Rootstock" in README and his…#179
ivegabr merged 1 commit intorsksmart:mainfrom
aapsi:patch-1

Conversation

@aapsi
Copy link
Copy Markdown
Contributor

@aapsi aapsi commented Jun 5, 2025

Summary

  • Corrected the example output in the README to show “Rootstock Testnet”

Output example:

- 🔍 Fetching transaction history on Rootstack Testnet for 0x19661D036D4e590948b9c00eef3807b88fBfA8e1 ...
+ 🔍 Fetching transaction history on Rootstock Testnet for 0x19661D036D4e590948b9c00eef3807b88fBfA8e1 ...
  • Corrected the history command message to reference “Rootstock” correctly:
- 🔍 Fetching transaction history on Rootstack ${
-   testnet ? "Testnet" : "Mainnet"
- } for ${walletAddress} ...
+ 🔍 Fetching transaction history on Rootstock ${
+   testnet ? "Testnet" : "Mainnet"
+ } for ${walletAddress} ...

ivegabr
ivegabr approved these changes Jun 16, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@ivegabr ivegabr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ivegabr ivegabr merged commit 39fe3bb into rsksmart:main Jun 16, 2025
3 checks passed
ezequiel-rodriguez
ezequiel-rodriguez reviewed Jun 19, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@ezequiel-rodriguez ezequiel-rodriguez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ezequiel-rodriguez
Copy link
Copy Markdown
Collaborator

ezequiel-rodriguez commented Jun 19, 2025

Thanks for the finding @aapsi ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ezequiel-rodriguez ezequiel-rodriguez ezequiel-rodriguez left review comments

@ivegabr ivegabr ivegabr approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aapsi @ezequiel-rodriguez @ivegabr